Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix WCT for bower: use Polymer version of sinon.js #3489

Closed
wants to merge 1 commit into from

Conversation

web-padawan
Copy link
Contributor

Fixes #3488

This is a fix for the critical issue that affects our bower installations:

bower sinonjs#^1.14.1                                ECMDERR Failed to execute "git ls-remote --tags --heads https://github.com/blittle/sinon.js.git", exit code of #128 remote: Invalid username or password. fatal: Authentication failed for 'https://github.com/blittle/sinon.js.git/'
Additional error details:
remote: Invalid username or password.
fatal: Authentication failed for 'https://github.com/blittle/sinon.js.git/'

Ping @justinfagnani @usergenic.

alvarezguille added a commit to vaadin/vaadin-ordered-layout that referenced this pull request Feb 10, 2020
Repository github.com/blittle/sinon.js was deleted
And bower install fails
Workaround for Polymer/tools#3488
While Polymer/tools#3489 isn't merged and released
web-padawan pushed a commit to vaadin/vaadin-ordered-layout that referenced this pull request Feb 10, 2020
Repository github.com/blittle/sinon.js was deleted
And bower install fails
Workaround for Polymer/tools#3488
While Polymer/tools#3489 isn't merged and released
DiegoCardoso added a commit to vaadin/vaadin-grid that referenced this pull request Feb 10, 2020
Repository github.com/blittle/sinon.js was deleted
And bower install fails
Workaround for Polymer/tools#3488
While Polymer/tools#3489 isn't merged and released
DiegoCardoso added a commit to vaadin/vaadin-grid that referenced this pull request Feb 10, 2020
Repository github.com/blittle/sinon.js was deleted
And bower install fails
Workaround for Polymer/tools#3488
While Polymer/tools#3489 isn't merged and released
@justinfagnani
Copy link
Contributor

The integration tests are timing out on Chrome 80. This has been happening elsewhere though, see lit/lit#1080 and lit/lit#1078

@web-padawan
Copy link
Contributor Author

@justinfagnani thanks. BTW, am I right that in order to make a release for Bower, this PR should go into (now archived) original web-component-tester repository?

At least bower search still points to that location:

$ bower search web-component-tester
Search results:

    web-component-tester https://github.com/Polymer/web-component-tester.git

@justinfagnani
Copy link
Contributor

@web-padawan I think you're right. cc @dfreedm @usergenic

@justinfagnani
Copy link
Contributor

We've unarchived the old repo and will add this fix there and tag a new release for Bower.

@web-padawan
Copy link
Contributor Author

Thanks a lot 🙇 Feel free to close this PR in that case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

web-component-tester fails to install: sinonjs repository not found
4 participants